Skip to content

chore: add live reload for web-js-sdk#1302

Open
asafshen wants to merge 2 commits intomainfrom
add-live-reload
Open

chore: add live reload for web-js-sdk#1302
asafshen wants to merge 2 commits intomainfrom
add-live-reload

Conversation

@asafshen
Copy link
Member

@asafshen asafshen commented Jan 6, 2026

Description

add live reload for web-js-sdk

Copilot AI review requested due to automatic review settings January 6, 2026 07:44
@vercel
Copy link

vercel bot commented Jan 6, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
access-key-management-widget Ready Ready Preview, Comment Jan 6, 2026 7:51am
audit-management-widget Ready Ready Preview, Comment Jan 6, 2026 7:51am
role-management-widget Ready Ready Preview, Comment Jan 6, 2026 7:51am
user-management-widget Ready Ready Preview, Comment Jan 6, 2026 7:51am
user-profile-widget Ready Ready Preview, Comment Jan 6, 2026 7:51am

@nx-cloud
Copy link

nx-cloud bot commented Jan 6, 2026

View your CI Pipeline Execution ↗ for commit 082cfc4

Command Status Duration Result
nx affected --target test:e2e ✅ Succeeded 1s View ↗
nx affected --target test ✅ Succeeded 2s View ↗
nx affected --target lint --fix=true ✅ Succeeded 2s View ↗
nx affected --target build ✅ Succeeded 3s View ↗
nx affected --target licenseCheck ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2026-01-06 14:32:25 UTC

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR introduces live reload functionality for the web-js-sdk to improve the development experience. The changes enable automatic browser refresh when source files or examples are modified, streamlining the development workflow.

Key changes:

  • Added a new Rollup configuration with BrowserSync for live reload during development
  • Created a dev npm script that builds and watches for changes
  • Fixed .env file path to work correctly with the new development server

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 4 comments.

File Description
packages/sdks/web-js-sdk/rollup.config.serve.mjs New Rollup configuration for development with BrowserSync, HTTPS support, and custom middleware to serve .env files
packages/sdks/web-js-sdk/package.json Added dev script that runs the new serve configuration with watch mode
packages/sdks/web-js-sdk/examples/loadEnv.js Fixed .env file path from /../.env to /.env to align with BrowserSync middleware
pnpm-lock.yaml Updated @descope/web-components-ui from 2.2.16 to 2.2.17, cleaned up duplicate TypeScript 5.4.5 snapshot entries
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Member

@ckiee ckiee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice (:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants